Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add Line of Sight (Visibility) system #87

Merged
merged 22 commits into from
Jan 17, 2018
Merged

Add Line of Sight (Visibility) system #87

merged 22 commits into from
Jan 17, 2018

Conversation

jonpas
Copy link
Member

@jonpas jonpas commented Jan 16, 2018 •
edited
Loading

  • Close Develop line of sight system #12
  • Functions for setting tile visibility effect
    • Reorder layers correctly
  • Add AETVObstacle class (opens Add Obstacles #86)
  • Add SensorRange property to Ship
    • Set SensorRange to MoveRange + 1 (will take proper effect when 9d4fb56 is merged)
  • Add tiled distance function
    • Restrict actions based on maximum distance they can be performed at
  • Use Vector2D for Tile reference in Actions
  • Make direct getters in GameModeBase in-line const
  • Set board visibility based on ship ranges
    • Hook actions into changing visible tiles
    • Block clicking on not visible enemy ships
    • Hide enemy ships from ship list
      • Remove fill on ship list initialization (manually call in begin play after visibility is all set)
    • Allow firing (Laser and Torpedo) through fog
      • Apply damage without logging if ship hit
    • Add debug toggles for ignoring visibility and showing enemy visibility during its turn
    • Add UpdateVisibleTiles() function which updates the board effects if called from player side or debug is enabled, and additionally returns array of visible tile positions (if passed, pass by ref)
    • Add GetVisibleShips() function which filters all ships on board by visibility (pass by ref)
  • Fix On(End/Begin)Perform() not being called in child classes
  • Increase map size to 50x50
  • Set all actions to end turn (at least for now - work on Project prototype finalization #83)
  • Resize Base grid tile to have 1px border (instead of 2px)
  • Change Hidden grid tile to black texture with 1px transparent border

Visibility Change on Move
Hidden Grid Tile

@jonpas
Initial work on hiding tile contents
dd96cd2
@jonpas
Merge branch 'master' into lineOfSight
62675d8
@jonpas
Merge branch 'master' into lineOfSight
93a8bad
@jonpas
Add Obstacle class, Add SensorRange property to Ship
1699e32
@jonpas
Merge branch 'master' into lineOfSight
d9a45f0
@jonpas
Merge branch 'master' into lineOfSight
72023f3
@jonpas
Merge branch 'master' into lineOfSight
6419df7
@jonpas
Add tiled distance and restrict actions based on maximum distance the…
be2afb7 
…y can be performed at
@jonpas
Use Vector2D for Tile reference in Actions
43cb628
@jonpas
Make direct Getters in GameModeBase in-line const
3a0ed2f
@jonpas
Set board visibility based on ship ranges, Hook actions into changing…
1d90c31 
… visible tiles, Set sensor ranges for ships, Fix On(End/Begin)Perform() not being called in child classes
@jonpas
Block clicking on not visible enemy ships
9f07837
@jonpas
Hide enemy ships from ship list, Remove fill on ship list initializat…
53f680c 
…ion (manually call in begin play after visibility is all set)
@jonpas
Allow firing (Laser and Torpedo) through fog, apply damage without lo…
888e0cc 
…gging if ship hit
@jonpas
Increase map size to 50x50, Set all actions to end turn (at least for…
409c99b 
… now - work on #83)
@jonpas
Add debug toggles for ignoring visibility and showing enemy visibilit…
135e10f 
…y during its turn
@jonpas
Resize Base grid tile to have 1px border (instead of 2px), Change Hid…
441b7f4 
…den grid tile to black texture with 1px transparent border
@jonpas
Update Game Mode Board with new properties
3306fc8
@jonpas jonpas added this to the Week 8 milestone Jan 16, 2018
@jonpas jonpas requested review from Nevith and planeer January 16, 2018 17:58
@jonpas jonpas merged commit d4974f1 into master Jan 17, 2018
@jonpas jonpas deleted the lineOfSight branch January 17, 2018 22:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nevith Nevith Nevith approved these changes

@planeer planeer planeer approved these changes

Assignees
No one assigned
Labels
art bug fix combat feature gameplay
Projects
None yet
Milestone
Week 8
Development

Successfully merging this pull request may close these issues.
3 participants
@jonpas @Nevith @planeer